Onboard C# Intellisense page to Unified Settings#71780
Conversation
…ause it's not in experiment
ghost
added
Area-IDE
untriaged
|
What are these: |
|
What is this:
|
| @@ -0,0 +1,226 @@ | |||
| { | |||
| "properties": { | |||
| "textEditor.csharp.intellisense.triggerCompletionOnTypingLetters": { | |||
There was a problem hiding this comment.
Can this be made more concise without losing descriptive power? I suggest replacing triggerCompletionOnTypingLetters with completeOnTyping. Or is the "letters" part important?
There was a problem hiding this comment.
Internally we have another option called triggerCompletionOnTyping
So this is called 'triggerCompletionOnTypingLetters' to avoid the misunderstanding.
…ne/roslyn into dev/shech/UnifiedSettings0
| } | ||
| }, | ||
| // CompletionOptionsStorage.SnippetsBehavior | ||
| "textEditor.csharp.intellisense.snippetsBehavior": { |
| @@ -0,0 +1,246 @@ | |||
| { | |||
There was a problem hiding this comment.
It feels like this file should be part of the MS.VS.LanguageServcie.CSharp project
There was a problem hiding this comment.
It needs to be deployed under /UnifiedSettings. Is this possible to do it if put under MS.VS.LanguageServcie.CSharp? We can address later if it's needed
7 participants
This is based on: Prerequisite: #71777DoneOnboard C# Intellisense page to Unified Settings. Once this is in, I can bring another PR to check the VB side settings in (would be very simple once this is in).
How it looks like:
Note: The *;..\Microsft.VisualStudio.LanguagesService.dll is a known bug in the platform. They will fix it soon.Updates: Find another bug in platform, so keep this as a draft until
https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/523746 is checked in